Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

docs(envoy): Document ports used by envoy #2825

Merged
merged 1 commit into from
Mar 15, 2021

Conversation

ksubrmnn
Copy link
Contributor

Signed-off-by: Kalya Subramanian kasubra@microsoft.com

Description:
Documents ports used by Envoy so that customers know not to use them for their applications.

Closes #2106

Affected area:

  • New Functionality [ ]
  • Documentation [X]
  • Install [ ]
  • Control Plane [ ]
  • CLI Tool [ ]
  • Certificate Management [ ]
  • Networking [ ]
  • Metrics [ ]
  • SMI Policy [ ]
  • Security [ ]
  • Tests [ ]
  • CI System [ ]
  • Performance [ ]
  • Other [ ]

Please answer the following questions with yes/no.

  • Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

Signed-off-by: Kalya Subramanian <kasubra@microsoft.com>
@ksubrmnn ksubrmnn requested a review from a team as a code owner March 12, 2021 18:41

## Ports
Do not use the following ports as they are used by the Envoy sidecar.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Do not use the following ports as they are used by the Envoy sidecar.
The following ports are used by the Envoy sidecar.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@draychev, I don't agree with the suggested wording, it is rather ambiguous when this section of the doc needs to be explicit.

@ksubrmnn ksubrmnn linked an issue Mar 12, 2021 that may be closed by this pull request
@ksubrmnn ksubrmnn merged commit 82a4639 into openservicemesh:main Mar 15, 2021
@ksubrmnn ksubrmnn deleted the ports branch March 15, 2021 21:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document User ID and ports used by OSM
5 participants